Skip to content

Conversation

@manuelkiessling
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
License MIT

Looks like some obsolete data access approach via a $data array was still in the docs.

@carsonbot carsonbot added Bug Bug Fix Status: Needs Review Needs to be reviewed labels May 6, 2024

$this->emit('productAdded', [
'product' => $product->getId(),
'productId' => $product->getId(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That will make the Product entity example (L2745-2752) not working :|

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you keep the "product" var name and fix the $data[...] you spotted ? :)

@smnandre smnandre added LiveComponent Documentation Improvements or additions to documentation and removed Bug Bug Fix labels May 6, 2024
@smnandre
Copy link
Member

Done in #2402

@smnandre smnandre closed this Dec 23, 2024
@manuelkiessling manuelkiessling deleted the patch-1 branch December 23, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Improvements or additions to documentation LiveComponent Status: Needs Review Needs to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants